-
Notifications
You must be signed in to change notification settings - Fork 4
Update non-major dependencies #50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
This change updates several key dependencies across both packages to their latest patch versions, ensuring compatibility and addressing minor improvements or fixes without altering end-user functionality. Walkthrough
Model: o4-mini | Prompt Tokens: 758 | Completion Tokens: 671 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We’re sharing gentle observations from an AI-powered code review to inspire improvements. These suggestions are meant as guidance, not strict requirements. Please pick and adapt what resonates with your workflow. You remain in control—your expertise and creative decisions shape the final results while AI offers support along the way.
Model: o4-mini | Prompt Tokens: 1330 | Completion Tokens: 3601
| "octokit": "5.0.3", | ||
| "parse-diff": "0.11.1", | ||
| "zod": "4.1.5" | ||
| "zod": "4.1.8" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You’re pinning zod to an exact patch version (4.1.8). To allow non-breaking updates and reduce future manual bumps, consider using a caret range for patch updates. For example:
{
"dependencies": {
"zod": "^4.1.8"
}
}| "octokit": "5.0.3", | ||
| "parse-diff": "0.11.1", | ||
| "zod": "4.1.5" | ||
| "zod": "4.1.8" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You’re pinning zod to an exact patch version (4.1.8). To allow non-breaking updates and reduce future manual bumps, consider using a caret range for patch updates. For example:
{
"dependencies": {
"zod": "^4.1.8"
}
}| "devDependencies": { | ||
| "@eslint/js": "9.35.0", | ||
| "@types/node": "22.18.1", | ||
| "@types/node": "22.18.3", | ||
| "@vercel/ncc": "0.38.3", | ||
| "eslint": "9.35.0", | ||
| "eslint-plugin-import": "2.32.0", | ||
| "eslint-plugin-sonarjs": "3.0.5", | ||
| "eslint-plugin-unicorn": "61.0.1", | ||
| "eslint-plugin-unicorn": "61.0.2", | ||
| "prettier": "3.6.2", | ||
| "tsx": "4.20.5", | ||
| "typescript": "5.9.2", | ||
| "typescript-eslint": "8.42.0", | ||
| "typescript-eslint": "8.43.0", | ||
| "yaml": "2.8.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same set of devDependencies is duplicated across multiple packages. You can DRY this up by moving common dev dependencies into the root package.json (if using a monorepo) and leveraging workspaces. For example:
// root package.json
{
"private": true,
"workspaces": ["pr-review", "pr-summary"],
"devDependencies": {
"@eslint/js": "9.35.0",
"@types/node": "^22.18.3",
"@vercel/ncc": "0.38.3",
"eslint": "9.35.0",
"eslint-plugin-import": "2.32.0",
"eslint-plugin-sonarjs": "3.0.5",
"eslint-plugin-unicorn": "^61.0.2",
"prettier": "3.6.2",
"tsx": "4.20.5",
"typescript": "5.9.2",
"typescript-eslint": "^8.43.0",
"yaml": "2.8.1"
}
}| "devDependencies": { | ||
| "@eslint/js": "9.35.0", | ||
| "@types/node": "22.18.1", | ||
| "@types/node": "22.18.3", | ||
| "@vercel/ncc": "0.38.3", | ||
| "eslint": "9.35.0", | ||
| "eslint-plugin-import": "2.32.0", | ||
| "eslint-plugin-sonarjs": "3.0.5", | ||
| "eslint-plugin-unicorn": "61.0.1", | ||
| "eslint-plugin-unicorn": "61.0.2", | ||
| "prettier": "3.6.2", | ||
| "tsx": "4.20.5", | ||
| "typescript": "5.9.2", | ||
| "typescript-eslint": "8.42.0", | ||
| "typescript-eslint": "8.43.0", | ||
| "yaml": "2.8.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same set of devDependencies is duplicated across multiple packages. You can DRY this up by moving common dev dependencies into the root package.json (if using a monorepo) and leveraging workspaces. For example:
// root package.json
{
"private": true,
"workspaces": ["pr-review", "pr-summary"],
"devDependencies": {
"@eslint/js": "9.35.0",
"@types/node": "^22.18.3",
"@vercel/ncc": "0.38.3",
"eslint": "9.35.0",
"eslint-plugin-import": "2.32.0",
"eslint-plugin-sonarjs": "3.0.5",
"eslint-plugin-unicorn": "^61.0.2",
"prettier": "3.6.2",
"tsx": "4.20.5",
"typescript": "5.9.2",
"typescript-eslint": "^8.43.0",
"yaml": "2.8.1"
}
}Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This PR contains the following updates:
22.18.1->22.18.361.0.1->61.0.28.42.0->8.43.04.1.5->4.1.8Release Notes
sindresorhus/eslint-plugin-unicorn (eslint-plugin-unicorn)
v61.0.2Compare Source
throw-new-error: Fix false positive for Effect library (#2740)0ddf5eano-useless-undefined: IgnoreSet#delete(undefined)(#2739)99acd27prefer-includesandprefer-array-index-of: Prevent crash on Vue files (#2645)a1da518typescript-eslint/typescript-eslint (typescript-eslint)
v8.43.0Compare Source
🩹 Fixes
❤️ Thank You
You can read about our versioning strategy and releases on our website.
colinhacks/zod (zod)
v4.1.8Compare Source
v4.1.7Compare Source
v4.1.6Compare Source
Configuration
📅 Schedule: Branch creation - "every weekend" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.